BSD Commit Log Search

[flang] Replace FlangOMPContext with OmpVariantMatchContext. (#209220)

The `FlangOMPContext` got replaced in 8e8e4e50f501b.

This fixes build after https://github.com/llvm/llvm-project/pull/206988.

[NFC][AMDGPU] Simplify scratch address materialization in SVS frame index fallback (#208624)

The flat scratch SVS frame index fallback folded the offset in with
V_ADD_U32_e32 writing into FrameReg (an SGPR, an illegal VALU
destination) and added it to FrameReg instead of the scavenged
VGPR used as vaddr, so the offset never reached the address.

This commit asserts that there is no frame register in this block, and
simplifies code accordingly. Correctly materialize the offset into the
scavenged VGPR and emit a scratch instruction in SVS form.

Add assert

[Polly] Cast delinearization terms to matching types (#206711)

#204146 revealed that Polly can call `SCEVDivision::divide` with a
numerator and denominator of different types. I believe such divisions
are not meaningful and should be avoided. As far as I can tell, the root
cause is that `SCEVRemoveMax::rewrite` may collect terms whose types
differ from that of the original expression.

Before this patch, the terms gathered by `SCEVRemoveMax::rewrite` were
used for delinearization as they are. This patch applies
`getTruncateOrSignExtend` to them to matching their types with that of
the original expression.

[Basic] Fix crash computing the spelling index of an unknown scoped attribute (#208909)

AttributeCommonInfo::getAttributeSpellingListIndex() aborts for an
unknown attribute that has a scope, such as [[ns::foo]]. It calls
calculateAttributeSpellingListIndex(), which runs name/scope
string-switches generated for the known attributes. Those switches
intentionally have no default: getScopeFromNormalizedScopeName() falls
off the end and hits llvm_unreachable ("Fell off the end of a
string-switch") for the unknown scope.

Any caller that asks a scoped unknown attribute for its spelling index
therefore crashes. An unrecognized attribute has no entry in those
tables and carries a single no-spelling, so short-circuit
calculateAttributeSpellingListIndex() for UnknownAttribute and return
index 0 before the switches run.

Add a unit test that constructs an unknown scoped AttributeCommonInfo
and asks for its spelling index; it aborts without the fix.

 Address more comments (const variables, std::optional PromVT)

[SystemZ][z/OS] Move emission of PPA1 (#208543)

There is an interesting restriction during emission: emitting debug
information closes the text section. As result, emitting the delayed
PPA1 data does not work in the current place together with emitting
debug information. Since the PPA1 information is constant data, the
solution is to emit the data in the target strreamer, in
`emitConstantPools()`.

[Flang][Fir] Set default alignment to 64 on inlineAllocation for arrays (#208781)

In https://github.com/llvm/llvm-project/pull/206525 the runtime was set
to default alloctable and pointer arrays to 64byte alignment. In this commit
the inlineallocation case is handled.

The same approach was taken and an alignment attribute was added,
with logic to promote to aligned_alloc or posixMemalign for apple
targets, when requested alignment is greater than 16.

Of note alignof(std::max_align_t)) was not used as the runtime target
may be different than the compile target, so 16 was hardcoded as when to
use aligned_alloc

Assisted-by: Opus 4.8

[clang] Reuse driver option strings in CompilerInvocation (#202632)

CompilerInvocation emits a second copy of clang/Options/Options.inc with
OPTTABLE_STR_TABLE_CODE only to implement lookupStrInTable(). The driver
OptTable already owns the identical string table.

On an arm64 Release build, fully stripped standalone clang shrinks from
94,570,224 to 94,537,216 bytes, saving 33,008 bytes (0.035%).

Work towards #202616

AI tool disclosure: Co-authored with OpenAI Codex.

Merge branch 'main' into users/kasuga-fj/polly-limit-delin-terms

[Hexagon] Add scs multilib for -fsanitize=shadow-call-stack (#208833)

musl hexagon targets already select msan/asan library variants based on
the sanitizer in use; do the same for shadow-call-stack so that
-fsanitize=shadow-call-stack picks up usr/lib/scs.

[ExpandReductions] Expand fmaximum/fminimum reductions (#195169)

fixes #195111

Address comments

[mlir][acc] Preserve shared data operations during if-clause lowering (#208771)

Example:
```fortran
!$acc kernels present(grid, c) if(offload_on)
do nc = 1, grid%rc%ncol
  c(1,nc) = 0
end do
c(1,1) = 0
!$acc end kernels
```

In this code, kernels restructuring can create compute constructs that
share data-entry operations with an enclosing data region. If-clause
lowering could rewrite or erase these shared operations, producing
invalid IR or duplicate erasure.

Fix: Preserve externally owned data operations, deduplicate cleanup, and
retain duplicate operand ordering while cloning local operations once.

Replace union with existing `PointerUnion`

[analyzer][docs] CSA release notes for clang-23 (#207433)

The commits were gathered using:
```sh
git log --reverse --oneline llvmorg-23-init..llvm/main \
  clang/lib/StaticAnalyzer clang/include/clang/StaticAnalyzer | \
  grep -v NFC | grep -v OpenACC | grep -v -i revert | grep -v -i "webkit"
```

I used the `LLVM_ENABLE_SPHINX=ON` and `LLVM_ENABLE_DOXYGEN=ON` cmake
options to enable the `docs-clang-html` build target, which generates
the html into `build/tools/clang/docs/html/ReleaseNotes.html` of which I
attach the screenshots to let you judge if it looks all good or not.

I've swapped the PR links to the issue links when a PR was fixing an
open GitHub issue. Those issues anyway have a link to the PR inside.

Assisted-by: Claude Opus 4.8

[lldb][test] Fix TestStatusline path resolution in symlinked environments (#209083)

Fixes a test timeout in TestStatusline.py introduced in commit
a35565161078 (#208609) when running in environments with symlinked
source paths.

[clang][OpenMP][test] Use -fopenmp=libomp explicitly in clang driver test (#209082)

Explicitly pass -fopenmp=libomp in the test to make it robust against
downstream configurations where the default OpenMP runtime is set to
something else (e.g., libgomp). Seems this is in alignment with other
tests in `clang/test/Driver`.

Remove new line.

[analyzer] Fix stale z3 analyzer-constraints name in test z3-logicalexpr-eval.c (#209185)

z3 was recently renamed to unsupported-z3.

Change the analyzer-constraints to unsupported-z3 in the test file
z3-logicalexpr-eval.c

Add test w/ static field

Address Comments

Use C++ standard layout instead

Revert earlier change

Add RUN lines to check Sema check works if AMDGCN is aux target

Factor out check for AMDGPU types so it can be reused.

Update implementation

Re-allow wrappers around named barriers

[clang][AMDGPU] Clean-up handling of named barrier type

- Do not allow the type in struct fields. This is more like a handle/resource than a real type. It does not follow the traditional C++ object model, and using it in a struct field can do some weird things if you instantiate too many of them.
- Use a `hip_barrier` LangAS for this type that currently maps to the local AS. This allows easy switching to the barrier AS in a future patch.

Alternative to #195612, see also #195613

[review] Improved parsing check.