[clang] [clang][codegen] Don't mark "int" TBAA on FP libcalls with indirect args (PR #108853)

Benjamin Maxwell via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 17 05:46:43 PDT 2024


MacDue wrote:

> > > How does this interact with #107598?
> > 
> > 
> > Though this also changes things to ensure when TBAA data is set, it's always set on the call.
> 
> Wasn't already doing that? (setting the TBAA on the call?).

It was setting it on `cast<llvm::Instruction>(Call.getScalarVal());` not the call (which you can get via an output on `EmitCall()`). At least in this case that meant it was putting the TBAA metadata on the `load x86_fp80` after the call. I'm not sure if there's other cases where something similar could happen. 


https://github.com/llvm/llvm-project/pull/108853


More information about the cfe-commits mailing list