[NFC][Attributor] Remove "See AbstractAttribute::..." comments
Most overridden methods have a comment like
`/// See AbstractAttribute::initialize(...).` that only points to the
base class declaration. Remove them, including those in OpenMPOpt and
AMDGPUAttributor.
This PR was assisted by AI but I reviewed all the changes.
RuntimeLibcalls: Add signext/zeroext attributes to libcall signatures
Try to match TargetLibraryInfo's behavior. Integer types narrower than
int are extended according to their signedness.
Co-authored-by: Claude Opus 5.5 <noreply at anthropic.com>
Hexagon: Remove kill flags on virtual registers from MIR tests (#236936)
Prepare to disallow this in the machine verifier. Move the one unique
case from qfp-remove-kill.mir into qfp-mix.mir; the rest only differed
from other tests in kill flags.
Co-authored-by: Claude Opus 5.5 <noreply at anthropic.com>
[NFC][clang][Driver] Changed Triple constructor (#236542)
This PR changes the constructor used to build the triple in the
`OffloadTargetInfo` constructor. It builds the `Triple` directly from
the joined string, instead of parsing it into a temporary and then
making a second `Triple` from its components. This avoids creating the
same object twice. This is an NFC change.
mail/spamassassin-devel: Enable two modules
Enable these modules:
- lib/Mail/SpamAssassin/URI.pm, used by Plugin::Redirectors
- lib/Mail/SpamAssassin/Util/Zip.pm, used by Plugin::OLEVBMacro
and Handler::Archive
RuntimeLibcalls: Add signext/zeroext attributes to libcall signatures
Try to match TargetLibraryInfo's behavior. Integer types narrower than
int are extended according to their signedness.
Co-authored-by: Claude Opus 5.5 <noreply at anthropic.com>
dsynth - Fix improper deletions with "prepare-system"
* Don't delete "obsolete" packages for "prepare-system"
* Obsolete packages can only be properly deleted when
a full scan is done. "prepare-system" only does a
partial scan so it winds up deleting stuff it
should not delete.
[Clang][CodeGen] Fix crash on break out of a for loop with a `C89` `init` declaration (#235809)
Fixes #170243
Since #156643 a variable declared in the init-statement of a C89 `for`
loop has no `LexicalScope` of its own, so its cleanup sits on the
enclosing scope's cleanup stack. `for.end` was still created before the
init-statement though, so its recorded cleanup depth didn't include that
cleanup. A `break` then had to be threaded through it, which left
`for.end` with no users, so it got deleted at the end of the loop while
the cleanup still pointed at it, and `FinishFunction` later emitted a
branch to the freed block.
In C89 mode the loop exit is now created after the init-statement, so it
records the right depth and leaving the loop branches to `for.end`
directly. That's the intended behaviour, the variable belongs to the
enclosing scope and its cleanup runs when that scope ends. The C99/C++
path is untouched.
AI Assisted.
RuntimeLibcalls: Add signext/zeroext attributes to libcall signatures
Try to match TargetLibraryInfo's behavior. Integer types narrower than
int are extended according to their signedness.
Co-authored-by: Claude Opus 5.5 <noreply at anthropic.com>