[PATCH] D69618: NeonEmitter: clean up prototype modifiers
Eli Friedman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 18 18:18:40 PST 2019
efriedma added a comment.
LGTM with a couple nits.
================
Comment at: clang/include/clang/Basic/arm_neon_incl.td:203
//
// The modifier 'd' means "default" and does not modify the base type in any
// way. The available modifiers are given below.
----------------
'd' is gone.
================
Comment at: clang/utils/convert_arm_neon.py:1
+#!/usr/bin/env python3
+
----------------
Are you going to commit this script? If you are, probably makes sense to include some sort of date, so it's clear which change you're talking about, and when it makes sense to remove it from the tree.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D69618/new/
https://reviews.llvm.org/D69618
More information about the cfe-commits
mailing list