This is an archive of the discontinued LLVM Phabricator instance.

[llvm][LowerConstantIntrinsics] add debug statements
ClosedPublic

Authored by nickdesaulniers on Feb 7 2023, 11:50 AM.

Details

Summary

These can help us better understand what builtin_constant_p and
builtin_object_size get folded to.

$ clang -mllvm -debug-only=lower-is-constant-intrinsic ...

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptFeb 7 2023, 11:50 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
nickdesaulniers requested review of this revision.Feb 7 2023, 11:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 7 2023, 11:50 AM
void accepted this revision.Feb 17 2023, 11:05 AM

These look fine, so accepted. One comment: Would it be good to somehow identify in the message the intrinsic each message is folding?

This revision is now accepted and ready to land.Feb 17 2023, 11:05 AM

These look fine, so accepted. One comment: Would it be good to somehow identify in the message the intrinsic each message is folding?

I think that's what II is in the message, right?

void added a comment.Mar 1 2023, 12:23 PM

These look fine, so accepted. One comment: Would it be good to somehow identify in the message the intrinsic each message is folding?

I think that's what II is in the message, right?

Doh! Of course. :-)